Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

fix(login): revert provider user ID format #112

Merged
merged 1 commit into from
Mar 1, 2024
Merged

fix(login): revert provider user ID format #112

merged 1 commit into from
Mar 1, 2024

Conversation

frgfm
Copy link
Member

@frgfm frgfm commented Mar 1, 2024

This PR fixes a typo in the format used for provider_user_id, which was causing a crash on the login route.

@frgfm frgfm added the type: fix label Mar 1, 2024
@frgfm frgfm added this to the 0.1.0 milestone Mar 1, 2024
@frgfm frgfm self-assigned this Mar 1, 2024
Copy link

codecov bot commented Mar 1, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 83.35%. Comparing base (4f48666) to head (3724a23).
Report is 1 commits behind head on main.

Files Patch % Lines
src/app/api/api_v1/endpoints/login.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   83.35%   83.35%           
=======================================
  Files          29       29           
  Lines         883      883           
=======================================
  Hits          736      736           
  Misses        147      147           
Flag Coverage Δ
unittests 83.35% <50.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@frgfm frgfm merged commit 0542e2d into main Mar 1, 2024
17 of 18 checks passed
@frgfm frgfm deleted the login-fix branch March 1, 2024 12:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant